-
Couldn't load subscription status.
- Fork 207
#3707 Refactor API endpoint to generate assistant messages with updated request and response structures #905
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#3707 Refactor API endpoint to generate assistant messages with updated request and response structures #905
Conversation
…uest and response structures
…w request structure and additional properties
684279f to
98477ac
Compare
98477ac to
a0a34c8
Compare
| # Mintlify Documentation | ||
|
|
||
| Install the [Mintlify CLI](https://www.npmjs.com/package/mintlify) to preview the documentation changes locally. To install, use the following command | ||
|
|
||
| ``` | ||
| npm i -g mintlify | ||
| ``` | ||
|
|
||
| Run the following command at the root of your documentation (where mint.json is) | ||
|
|
||
| ``` | ||
| mintlify dev | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Open source repo so I thought fit to give it at least a bit of a README lol. More effort might be warranted for this at some point in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one small q before signing off!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to dogfood our own features, we might want to add a new tag here: https://mintlify.com/docs/pages#param-tag
defer to @ethanpalm if we actually want this or not though!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to add that if desired!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can totally merge without for now but depending on Ethan's preference we can add it in a follow-up :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great!
Documentation changes
Changes the discovery API openapi spec to remove the old
topicandmessageroutes and add the newGenerate assistant messageroute.Closes half of https://linear.app/mintlify/issue/ENG-3707/update-chat-endpoints-to-single-assistant-endpoint
For Reviewers
When reviewing documentation PRs, please consider:
✅ Technical accuracy
✅ Clarity and completeness (N/A since this is just an api request)
✅ User experience